Remove storage sync services in resource cleanup#14196
Conversation
benbp
added
the
Central-EngSys
There was a problem hiding this comment.
Pull request overview
Adds Storage Sync Service cleanup to prevent leaked Storage Sync resources from blocking resource group deletion during test resource teardown.
Changes:
- Introduces
Remove-StorageSyncServiceshelper to remove Storage Sync Services (and dependent sync groups/endpoints) prior to RG deletion. - Invokes the new helper from
Remove-TestResources.ps1during resource cleanup. - Updates default subscription selection messaging/behavior and adds a well-known subscription entry.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| eng/common/scripts/Helpers/Resource-Helpers.ps1 | Adds helper that enumerates Storage Sync Services and deletes endpoints/groups/services to unblock RG deletion. |
| eng/common/TestResources/Remove-TestResources.ps1 | Calls the new helper during cleanup; adjusts subscription selection defaults and subscription name map. |
|
The following pipelines have been queued for testing: |
benbp
force-pushed
the
benbp/delete-storage-sync
branch
from
b87ac23 to
c67206f
Compare
|
The following pipelines have been queued for testing: |
benbp
force-pushed
the
benbp/delete-storage-sync
branch
from
c67206f to
324eb63
Compare
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#14196 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#14196 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Remove storage sync services in resource cleanup